Skip to content

BLD: Updading travis scipy package.#960

Merged
bblay merged 2 commits into
SciTools:masterfrom
cpelley:latest_scipy
Jan 22, 2014
Merged

BLD: Updading travis scipy package.#960
bblay merged 2 commits into
SciTools:masterfrom
cpelley:latest_scipy

Conversation

@cpelley

@cpelley cpelley commented Jan 17, 2014

Copy link
Copy Markdown

No description provided.

@cpelley

cpelley commented Jan 20, 2014

Copy link
Copy Markdown
Author

ON HOLD, currently CF site not accessible

@ghost ghost assigned bblay Jan 21, 2014
@bblay

bblay commented Jan 21, 2014

Copy link
Copy Markdown
Contributor

Ok, @cpelley, should be able to rebase onto master now travis is no longer affected by that website outage.

Comment thread .travis.yml Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These lines could do with a comment explaining why they are in this position in the file. (And hence when they might be able to be folded back into the normal section.)

@bblay

bblay commented Jan 22, 2014

Copy link
Copy Markdown
Contributor

Hopefully just a small change required to version number handling, fingers crossed.

@bblay

bblay commented Jan 22, 2014

Copy link
Copy Markdown
Contributor

Please could the benefits of switching to SciPy 0.12 be outlined here?
Was Travis getting 0.9 before this change?

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Benefits not understood by me, I don't think there is a specific user case driving it.

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Was Travis getting 0.9 before this change?

yes v0.9 I believe since travis is a 12.04 release of ubuntu (precise)

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Assuming that the above indicates a problem with the numpy version??
Making a new commit which prints out the matplotlib and numpy versions.

@bblay

bblay commented Jan 22, 2014

Copy link
Copy Markdown
Contributor

@rhattersley, what are your thoughts on building a version of SciPy for Travis and hosting it somewhere?

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Great, now uses the same numpy but with scipy 0.12.0

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Works

@bblay

bblay commented Jan 22, 2014

Copy link
Copy Markdown
Contributor

Great, now uses the same numpy but with scipy 0.12.0

Is that a problem?

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Great, now uses the same numpy but with scipy 0.12.0

Is that a problem?

No the opposite, using numpy from 12.04 ubuntu and scipy 0.12.0 built to use dependencies in 12.04

Thanks @esc24 for pointing us to pylab
https://launchpad.net/~pylab/+archive/stable

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Just rebasing now

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Passes, ready to go in.

Comment thread .travis.yml Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's wrong with sudo add-apt-repository -y ppa:pylab/stable?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll give it a go

@bblay

bblay commented Jan 22, 2014

Copy link
Copy Markdown
Contributor

Please hold off merging this while to possibility of upgrading to NumPy 1.7 is explored.

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Please hold off merging this while to possibility of upgrading to NumPy 1.7 is explored.

Since this would be from a separate PPA, I think that should go in a separate PR

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Found one here:
https://launchpad.net/~tukss/+archive/ppa/+build/4055943

Its unofficial but hey, its travis and it gets wiped so we can give it a go if you like?

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

(the one we have added for scipy is also unofficial so....)

@bblay bblay mentioned this pull request Jan 22, 2014
@bblay

bblay commented Jan 22, 2014

Copy link
Copy Markdown
Contributor

@cpelley I'm trying pip, can't do any harm for you to try launchpad, perhaps in another branch?

@bblay

bblay commented Jan 22, 2014

Copy link
Copy Markdown
Contributor

pip install -I numpy==1.7 took Travis about three minutes and the tests pass...

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Since travis already takes 23min... is 3 minutes significant?

@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

I'll try with the one off launchpad

bblay added a commit that referenced this pull request Jan 22, 2014
BLD: Updading travis scipy package.
@bblay bblay merged commit 132629f into SciTools:master Jan 22, 2014
@cpelley cpelley deleted the latest_scipy branch January 22, 2014 14:06
@cpelley

cpelley commented Jan 22, 2014

Copy link
Copy Markdown
Author

Update travis numpy -> v1.7.0b2
#975

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants